Reviewed 8 of 8 files at r1.
Reviewable status: 6 of 8 files reviewed, 2 unresolved discussions (waiting on @kormat and @worxli)

.buildkite/clean_env.sh, line 6 at r1 (raw file):

[ -n "$cntrs" ] && { echo "Remove left over containers: $cntrs"; docker rm -f "$cntrs"; }

echo "Try to remove left over networks"

Why try? Can it fail?

.buildkite/pipeline.sh, line 14 at r1 (raw file):

# begin the pipeline.yml file
".buildkite/common.sh"

Add a comment to note that common.sh sets $BASE, so it can't be used here yet.

Reviewable status: 3 of 7 files reviewed, 2 unresolved discussions (waiting on @kormat)

.buildkite/clean_env.sh, line 6 at r1 (raw file):

No, but there may be none.

.buildkite/pipeline.sh, line 14 at r1 (raw file):

Other way round.

Reviewed 5 of 5 files at r3.
Reviewable status: complete! all files reviewed, all discussions resolved